8c8124ce02fcedd77c9fdea04a19f4f7a38c7284,core/src/main/java/tachyon/worker/netty/DataServerHandler.java,DataServerHandler,validateBounds,#BlockRequest#number#,105

Before Change


  }

  private void validateBounds(final BlockRequest req, final long fileLength) {
    if (req.getOffset() > fileLength) {
      final String msg =
          String.format("Offset(%d) is larger than file length(%d)", req.getOffset(), fileLength);
      throw new IllegalArgumentException(msg);
    }
    if (req.getLength() != -1 && req.getOffset() + req.getLength() > fileLength) {
      final String msg =

After Change


  }

  private void validateBounds(final BlockRequest req, final long fileLength) {
    Preconditions.checkArgument(req.getOffset() <= fileLength,
        "Offset(%s) is larger than file length(%s)", req.getOffset(), fileLength);
    Preconditions.checkArgument(req.getLength() == -1
        || req.getOffset() + req.getLength() <= fileLength,
        "Offset(%s) plus length(%s) is larger than file length(%s)", req.getOffset(),